Skip to content

Conversation

@BranchS
Copy link
Contributor

@BranchS BranchS commented Jan 6, 2026

No description provided.

@BranchS
Copy link
Contributor Author

BranchS commented Jan 6, 2026

I'm not sure why those old commits are showing here... the change I made was to UPS (see changed files), which now supports 2Fa

UPS now supports 2fa

Signed-off-by: Branch <[email protected]>
@Carlgo11
Copy link
Member

Carlgo11 commented Jan 8, 2026

Hey @BranchS! I cleaned up your pull request so it only contains the UPS commit 😉

Are you sure SMS 2FA is available? I just went through the 2FA enrollment process and I never got the option to choose that 🤔

@Carlgo11 Carlgo11 added the add site Issue/PR adds a site to the repo. label Jan 8, 2026
@BranchS
Copy link
Contributor Author

BranchS commented Jan 8, 2026

Thank you! I’m not sure. I don’t have a phone number on my account so I can’t test. I did find this on the web: https://www.reddit.com/r/UPS/comments/1lmzvhp/any_idea_why_i_have_to_text_my_password_to_this/

but I’m not sure if that’s on the account level. If there’s no evidence of sms, we could remove that and still add the rest. Thanks!

@github-actions github-actions bot added the inactive Inactive issue/PR, soon to be closed unless the author responds. label Jan 15, 2026
@2factorauth 2factorauth deleted a comment from github-actions bot Jan 15, 2026
@Carlgo11 Carlgo11 removed the inactive Inactive issue/PR, soon to be closed unless the author responds. label Jan 15, 2026
Removed 'sms' from the two-factor authentication methods.

Signed-off-by: Carl <[email protected]>
@Carlgo11 Carlgo11 merged commit 6db3803 into 2factorauth:master Jan 20, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add site Issue/PR adds a site to the repo.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants